Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

powershell/7.4.2 package update #16744

Closed
wants to merge 2 commits into from

Conversation

octo-sts[bot]
Copy link
Contributor

@octo-sts octo-sts bot commented Apr 11, 2024

@octo-sts octo-sts bot added request-version-update request for a newer version of a package automated pr labels Apr 11, 2024
@debasishbsws debasishbsws self-assigned this Apr 22, 2024
@debasishbsws
Copy link
Member

debasishbsws commented Apr 22, 2024

Looks like the dotnet sdk has another release but that is not included in the dotnet monolith repository we are using here.

@debasishbsws
Copy link
Member

In the official release we have more than one SDK version https://dotnet.microsoft.com/en-us/download/dotnet/8.0

Should we break the dotnet package into dotnet-runtime and dotnet-sdk

@debasishbsws
Copy link
Member

Though the docs suggest to build dotnet v8+ from the repo dotnet/dotnet but we can use the dotnet/installer which we were using for dotnet-6 and dotnet-7

@debasishbsws
Copy link
Member

This will take time as it may need #17580

@debasishbsws
Copy link
Member

It will get Fix after #17699 get merged into main

@TaylorBloom129
Copy link
Contributor

@rawlingsj to follow up with Dimitri

@debasishbsws
Copy link
Member

The issue for powershell build failure is connected to #17580 Dotnet sdk version mismatch it requires version 8.0.204 but in wolfi we have dotnet-8-sdk 8.0.104.

I have tried to separate the SDK from the main package in #17699 and the powershell is building perfectly with that. But there will be incompatibility issue.

Yesterday I tried to build the dotnet-8 package from the dotnet/installer repo as they provide the new SDK v8.0.204 but could not able to build it correctly.

@rawlingsj rawlingsj added the help wanted Extra attention is needed label May 2, 2024
@rawlingsj
Copy link
Member

Ok thanks @debasishbsws, I don't have any knowledge of powershell so we might need some help from @xnox or @smoser 🙏

@debasishbsws debasishbsws removed their assignment May 2, 2024
@lambdaclan
Copy link

Hello there. I am not sure if this is the right place to report this, but I would like to bring an issue related to the PowerShell package to your attention. I am relatively new to using Wolfi images in containers, so I am not 100% familiar of the relation between Wolfi and Alpine packages, but it seems like the Wolfi PowerShell package is incomplete in the same fashion the Alpine package is.

Basically, the provided PowerShell package is missing quite a few of the built-in modules which in turn significantly reduces usability of the shell. There is already an upstream issue here PowerShell/PowerShell#20143.

The good news is that there is a workaround. If you install PowerShell via dotnet as a global tool then all modules become available.

To reproduce the issue you can use the following as a reference:

apk add powershell
pwsh
Get-Module -ListAvailable

image

apk add dotnet-8-sdk
dotnet tool install --global PowerShell
~/.dotnet/tools/pwsh
Get-Module -ListAvailable

image

I am not really sure what the best course of action is at this moment, maybe just wait for upstream fix?

If I need to create a proper issue under a different repo or something please let me know.

Thank you.

@rawlingsj
Copy link
Member

Thanks @lambdaclan! Appreciate the info. We do have an issue tracker on this repo but given we have a problem with powershell with this PR let's see if we can get some thoughts from others.

@smoser don't suppose you got anywhere with looking at Powershell?

wolfi-bot and others added 2 commits June 6, 2024 19:08
@krishjainx krishjainx force-pushed the wolfictl-552965a5-e7c2-4ca5-9c9e-d1e0152ba074 branch from 7affa04 to 2a4b749 Compare June 6, 2024 23:08
@octo-sts octo-sts bot closed this Jun 19, 2024
Copy link
Contributor Author

octo-sts bot commented Jun 19, 2024

superseded by #22270

@octo-sts octo-sts bot deleted the wolfictl-552965a5-e7c2-4ca5-9c9e-d1e0152ba074 branch June 20, 2024 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automated pr help wanted Extra attention is needed request-version-update request for a newer version of a package squad:foundations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants